chore: editing logs configuration [PAGOPA-2221] #130

FedericoRuzzier · 2024-10-02T14:35:08Z

List of Changes

Adding logs configuration
Configuring majority of api logs to debug

Motivation and Context

Needed in order to refactor logs and reduce costs

How Has This Been Tested?

Manually

Screenshots (if appropriate):

Types of changes

Bug fix (non-breaking change which fixes an issue)
New feature (non-breaking change which adds functionality)
Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

My change requires a change to the documentation.
I have updated the documentation accordingly.

github-actions · 2024-10-02T14:35:20Z

The default action is to increase the PATCH number of SEMVER. Set IGNORE-FOR-RELEASE if you want to skip SEMVER bump. BREAKING-CHANGE and NEW-RELEASE must be run from GH Actions section manually.

github-actions · 2024-10-02T14:35:21Z

The default action is to increase the PATCH number of SEMVER. Set IGNORE-FOR-RELEASE if you want to skip SEMVER bump. BREAKING-CHANGE and NEW-RELEASE must be run from GH Actions section manually.

src/main/java/it/gov/pagopa/wispconverter/controller/ReceiptController.java

src/main/java/it/gov/pagopa/wispconverter/controller/ReceiptTimerController.java

src/main/java/it/gov/pagopa/wispconverter/controller/RecoveryController.java

src/main/java/it/gov/pagopa/wispconverter/service/RecoveryService.java

src/main/java/it/gov/pagopa/wispconverter/controller/ReceiptTimerController.java

…onverter into PAGOPA-2221

# Conflicts: # src/main/java/it/gov/pagopa/wispconverter/controller/ReceiptController.java # src/main/java/it/gov/pagopa/wispconverter/controller/RecoveryController.java # src/main/java/it/gov/pagopa/wispconverter/scheduler/RecoveryScheduler.java # src/main/java/it/gov/pagopa/wispconverter/service/RPTTimerService.java # src/main/java/it/gov/pagopa/wispconverter/servicebus/ECommerceHangTimeoutConsumer.java # src/main/java/it/gov/pagopa/wispconverter/servicebus/PaymentTimeoutConsumer.java # src/main/java/it/gov/pagopa/wispconverter/servicebus/RPTTimeoutConsumer.java # src/main/java/it/gov/pagopa/wispconverter/servicebus/RTConsumer.java

…onverter into PAGOPA-2221

src/main/java/it/gov/pagopa/wispconverter/controller/RPTTimerController.java

src/main/java/it/gov/pagopa/wispconverter/controller/RecoveryController.java

+            log.debug("Invoking API operation recoverReceiptKOForCreditorInstitution - args: {} {} {}",
+                    sanitizeInput(ci), dateFrom, dateTo);


To fix the log injection issue, we need to sanitize the dateFrom and dateTo parameters before logging them. This can be done by removing any potentially harmful characters, such as new-line characters, which can be used to manipulate log entries. We will use a simple sanitization method that replaces new-line characters with an empty string.

Create a sanitization method to clean the dateFrom and dateTo parameters.

Apply this sanitization method to dateFrom and dateTo before logging them.

src/main/java/it/gov/pagopa/wispconverter/controller/RecoveryController.java

+            log.debug("Invoking API operation recoverReceiptKOForCreditorInstitution - args: {} {} {} {}",
+                    sanitizeInput(ci), sanitizeInput(iuv), dateFrom, dateTo);


To fix the log injection issue, we need to sanitize the dateFrom parameter before it is logged. This can be done by using the existing sanitizeInput method, which is already used for other parameters in the same method. This ensures that any potentially harmful characters are removed or escaped, preventing log injection.

General Fix: Sanitize user input before logging it.

Detailed Fix: Apply the sanitizeInput method to the dateFrom parameter in the recoverReceiptKOForCreditorInstitutionAndIUV method.

Specific Changes: Modify line 77 to use sanitizeInput(dateFrom) instead of dateFrom.

Requirements: No new methods or imports are needed as the sanitizeInput method is already available.

src/main/java/it/gov/pagopa/wispconverter/controller/RecoveryController.java

src/main/java/it/gov/pagopa/wispconverter/service/RPTTimerService.java

+
+        // build the service bus message
+        ServiceBusMessage serviceBusMessage = new ServiceBusMessage(message.toString());
+        log.debug("Sending scheduled message {} to the queue: {}", message.toString(), queueName);


To fix the log injection issue, we need to sanitize the user input before logging it. This can be done by removing or encoding any potentially harmful characters from the message object before it is logged. Specifically, we can replace newline characters and other special characters that could be used for log injection.

General Fix Approach:

Sanitize the message object before logging it.

Use a utility method to remove or encode potentially harmful characters.

Detailed Fix:

Create a utility method to sanitize the message object.

Use this utility method to sanitize the message before logging it in the sendMessage method of RPTTimerService.

Specific Changes:

Add a utility method sanitizeForLogging in the CommonUtility class.

Use this method to sanitize the message object before logging it in the sendMessage method.

src/main/java/it/gov/pagopa/wispconverter/service/RPTTimerService.java

+        Long sequenceNumber = serviceBusSenderClient.scheduleMessage(serviceBusMessage, scheduledExpirationTime);
+
+        // log event
+        log.debug("Sent scheduled message_base64 {} to the queue: {}", LogUtils.encodeToBase64(message.toString()), queueName);


To fix the log injection issue, we need to sanitize the user input before logging it. This can be done by removing or escaping any potentially harmful characters from the input. Specifically, we should replace newline characters and other control characters that could be used to manipulate log entries.

General Fix: Sanitize the user input to remove or escape harmful characters before logging.

Detailed Fix: Use a utility method to sanitize the message.toString() output before passing it to the LogUtils.encodeToBase64 method and logging it.

Specific Changes:

Add a utility method to sanitize the input string.

Use this utility method to sanitize the message.toString() output before logging.

github-actions · 2024-10-17T14:37:24Z

This PR exceeds the recommended size of 400 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size.

sonarcloud · 2024-10-17T14:38:25Z

Quality Gate failed

Failed conditions
66.2% Coverage on New Code (required ≥ 80%)
3.1% Duplication on New Code (required ≤ 3%)

See analysis details on SonarCloud

src/main/java/it/gov/pagopa/wispconverter/controller/RPTTimerController.java

src/main/java/it/gov/pagopa/wispconverter/controller/ReceiptTimerController.java

src/main/java/it/gov/pagopa/wispconverter/controller/RecoveryController.java

…/pagopa-wisp-converter into PAGOPA-2221 # Please enter a commit message to explain why this merge is necessary, # especially if it merges an updated upstream into a topic branch. # # Lines starting with '#' will be ignored, and an empty message aborts # the commit.

src/main/java/it/gov/pagopa/wispconverter/controller/RPTTimerController.java

@@ -51,9 +51,9 @@
    @Trace(businessProcess = RPT_BP_TIMER_SET, reEnabled = true)
    public void createTimer(@RequestBody RPTTimerRequest request) {
        try {
-            log.info("Invoking API operation createRPTTimer - args: {}", sanitizeInput(request.toString()));
+            log.debug("Invoking API operationcreateRPTTimer - args: {}", request.toString());


To fix the log injection issue, we need to sanitize the user input before logging it. This can be done by ensuring that the toString() method of the request object does not contain any characters that could be used for log injection, such as newlines or special characters. We can achieve this by using a utility method to sanitize the output of toString() before logging it.

Create a utility method to sanitize the string by removing or escaping potentially harmful characters.

Use this utility method to sanitize the output of request.toString() before logging it.

src/main/java/it/gov/pagopa/wispconverter/controller/ReceiptController.java

@@ -102,9 +106,9 @@
    public void receiptKo(@RequestBody String request) throws Exception {

        try {
-            log.info("Invoking API operation receiptKo - args: {}", request);
+            log.debug("Invoking API operationreceiptKo - args: {}", request);


To fix the log injection issue, we need to sanitize the user-provided input before logging it. This can be done by removing or escaping any potentially harmful characters such as newlines. We can use a utility method to sanitize the input string.

General Fix: Sanitize the user input to remove or escape harmful characters before logging.

Detailed Fix: Implement a utility method to sanitize the input by replacing newline characters with a space or another harmless character. Use this method to sanitize the request parameter before logging it.

Specific Changes: Modify the receiptKo method in src/main/java/it/gov/pagopa/wispconverter/controller/ReceiptController.java to sanitize the request parameter before logging.

src/main/java/it/gov/pagopa/wispconverter/controller/ReceiptTimerController.java

@@ -52,9 +54,9 @@
    @Trace(businessProcess = BP_TIMER_SET, reEnabled = true)
    public void createTimer(@RequestBody ReceiptTimerRequest request) {
        try {
-            log.info("Invoking API operation createTimer - args: {}", request.toString());
+            log.debug("Invoking API operationcreateTimer - args: {}", request.toString());


To fix the problem, we need to avoid logging sensitive information directly. Instead of logging the entire request object, we should log only non-sensitive information or a sanitized version of the request. We can create a custom method to generate a sanitized string representation of the ReceiptTimerRequest object, excluding sensitive fields.

src/main/java/it/gov/pagopa/wispconverter/controller/ReceiptTimerController.java

@@ -52,9 +54,9 @@
    @Trace(businessProcess = BP_TIMER_SET, reEnabled = true)
    public void createTimer(@RequestBody ReceiptTimerRequest request) {
        try {
-            log.info("Invoking API operation createTimer - args: {}", request.toString());
+            log.debug("Invoking API operationcreateTimer - args: {}", request.toString());


To fix the log injection issue, we need to ensure that any user-provided data included in log entries is properly sanitized. This can be achieved by sanitizing the output of the toString() method before logging it. We will use the existing sanitizeInput method to clean the data.

Modify the log statement on line 57 to sanitize the output of request.toString().

Ensure that the sanitizeInput method is used to clean the data before it is logged.

src/main/java/it/gov/pagopa/wispconverter/controller/ReceiptTimerController.java

@@ -77,10 +79,10 @@
    @Trace(businessProcess = BP_TIMER_DELETE, reEnabled = true)
    public void deleteTimer(@RequestParam() String paymentTokens) {
        try {
-            log.info("Invoking API operation deleteTimer - args: {}", paymentTokens);
+            log.debug("Invoking API operationdeleteTimer - args: {}", sanitizeInput(paymentTokens));


To fix the problem, we should avoid logging the paymentTokens directly. Instead, we can log a generic message indicating that the operation was invoked without including the sensitive data. This approach ensures that no sensitive information is written to the log files while still providing useful debug information.

Remove the logging of paymentTokens in the deleteTimer method.

Log a generic message indicating the invocation of the operation.

src/main/java/it/gov/pagopa/wispconverter/controller/RecoveryController.java

@@ -100,7 +102,7 @@
    @PostMapping(value = "/receipts")
    public ResponseEntity<RecoveryReceiptReportResponse> recoverReceiptToBeReSent(@RequestBody RecoveryReceiptRequest request) {
        try {
-            log.info("Invoking API operation recoverReceiptToBeReSent - args: {}", sanitizeInput(request.toString()));
+            log.debug("Invoking API operationrecoverReceiptToBeReSent - args: {}", request.toString());


To fix the log injection issue, we need to sanitize the user input before logging it. This can be done by replacing potentially harmful characters with safe alternatives. Specifically, we should remove or replace newline characters and other special characters that could be used to manipulate log entries.

The best way to fix this problem without changing existing functionality is to create a utility method that sanitizes the input and use this method before logging the request object. This ensures that any potentially harmful characters are removed or replaced.

src/main/java/it/gov/pagopa/wispconverter/controller/RecoveryController.java

    @Operation(summary = "Execute reconciliation for passed receipts by partition.", description = "Execute reconciliation of all receipts contained in the partitions of the request", security = {@SecurityRequirement(name = "ApiKey")}, tags = {"Recovery"})
    @ApiResponses(value = {
            @ApiResponse(responseCode = "200", description = "Reconciliation scheduled")
    })
    @PostMapping(value = "/partitions")
    public ResponseEntity<RecoveryReceiptReportResponse> recoverReceiptToBeReSentByPartition(@RequestBody RecoveryReceiptByPartitionRequest request) {
        try {
-            log.info("Invoking API operation recoverReceiptToBeReSentByPartition - args: {}", sanitizeInput(request.toString()));
+            log.debug("Invoking API operationrecoverReceiptToBeReSentByPartition - args: {}", request.toString());


To fix the log injection issue, we need to sanitize the user input before logging it. This can be done by replacing potentially dangerous characters in the user input with safe alternatives. Specifically, we should remove or replace newline characters and other special characters that could be used to manipulate log entries.

The best way to fix this problem without changing existing functionality is to sanitize the request object before logging it. We can create a utility method to sanitize the input and use this method in the log statement.

…/pagopa-wisp-converter into PAGOPA-2221 # Please enter a commit message to explain why this merge is necessary, # especially if it merges an updated upstream into a topic branch. # # Lines starting with '#' will be ignored, and an empty message aborts # the commit.

FedericoRuzzier added 2 commits October 2, 2024 15:27

PAGOPA-2221 changing logs to debug level

3c20e2a

PAGOPA-2221 adding logs config

b506b7d

FedericoRuzzier added enhancement New feature or request patch labels Oct 2, 2024

FedericoRuzzier requested a review from a team as a code owner October 2, 2024 14:35

auto-assign bot assigned FedericoRuzzier Oct 2, 2024

github-advanced-security bot found potential problems Oct 2, 2024

View reviewed changes

FedericoRuzzier added 2 commits October 2, 2024 16:41

PAGOPA-2221 mergin main

e367aa5

PAGOPA-2221 sanitize input

e83cc43

github-advanced-security bot found potential problems Oct 2, 2024

View reviewed changes

src/main/java/it/gov/pagopa/wispconverter/controller/ReceiptTimerController.java Fixed Show fixed Hide fixed

src/main/java/it/gov/pagopa/wispconverter/controller/ReceiptTimerController.java Fixed Show fixed Hide fixed

FedericoRuzzier changed the title ~~Pagopa 2221~~ Pagopa 2221 editing logs configuration Oct 2, 2024

PAGOPA-2221 update dockerfile

ad51483

FedericoRuzzier changed the title ~~Pagopa 2221 editing logs configuration~~ chore: editing logs configuration Pagopa 2221 Oct 3, 2024

Merge branch 'main' into PAGOPA-2221

736431e

aomegax marked this pull request as draft October 8, 2024 10:46

FedericoRuzzier and others added 6 commits October 15, 2024 12:42

[PAGOPA-2221] opt log conf

0a3a615

Merge branch 'PAGOPA-2221' of https://github.com/pagopa/pagopa-wisp-c…

c31f6f0

…onverter into PAGOPA-2221

[PAGOPA-2221] service bus opt log

3613ef6

[PAGOPA-2221] logging aspect

b774439

Merge branch 'PAGOPA-2221' of https://github.com/pagopa/pagopa-wisp-c…

3bd8df8

…onverter into PAGOPA-2221

github-actions bot added the size/large label Oct 16, 2024

jacopocarlini marked this pull request as ready for review October 16, 2024 10:40

jacopocarlini and others added 5 commits October 16, 2024 12:43

Delete .classpath

0655e05

[PAGOPA-2221] typo

b1d6afc

Merge branch 'PAGOPA-2221' of https://github.com/pagopa/pagopa-wisp-c…

1b53e4d

…onverter into PAGOPA-2221

[PAGOPA-2221] removed useless property

3e09e58

[PAGOPA-2221] removed sanitization from some data

b7d6cb0

Merge branch 'PAGOPA-2221' of https://github.com/pagopa/pagopa-wisp-c…

47de307

…onverter into PAGOPA-2221

github-advanced-security bot found potential problems Oct 16, 2024

View reviewed changes

alessio-acitelli and others added 2 commits October 17, 2024 08:59

Merge branch 'main' into PAGOPA-2221

6e2acfd

[PAGOPA-2221] chore(sampling): custom sampling

34b88a5

jacopocarlini changed the title ~~chore: editing logs configuration Pagopa 2221~~ chore: editing logs configuration [PAGOPA-2221] Oct 17, 2024

Bump to version 0.4.11-1-PAGOPA-2221 [skip ci]

061749c

github-advanced-security bot found potential problems Oct 17, 2024

View reviewed changes

pagopa-github-bot and others added 8 commits October 17, 2024 14:56

Bump to version 0.4.11-2-PAGOPA-2221 [skip ci]

67990d2

[PAGOPA-2221] fix

338656d

Bump to version 0.4.11-3-PAGOPA-2221 [skip ci]

c54c6b2

[PAGOPA-2221] fix

73cc92b

Bump to version 0.4.11-4-PAGOPA-2221 [skip ci]

1a47493

[PAGOPA-2221] fix

faa91d3

github-advanced-security bot found potential problems Oct 17, 2024

View reviewed changes

pagopa-github-bot and others added 4 commits October 17, 2024 16:14

Bump to version 0.4.11-5-PAGOPA-2221 [skip ci]

cfe5681

[PAGOPA-2221] fix

1d2d651

Bump to version 0.4.11-6-PAGOPA-2221 [skip ci]

813e689

aomegax assigned alessio-acitelli Oct 18, 2024

aomegax marked this pull request as draft October 18, 2024 10:37

aomegax assigned jacopocarlini Oct 18, 2024

@@ -48,3 +48,3 @@
                         log.debug("Invoking API operationrecoverReceiptKOForCreditorInstitution - args: {} {} {}",
-                                sanitizeInput(ci), dateFrom, dateTo);
+                                sanitizeInput(ci), sanitizeInput(dateFrom), sanitizeInput(dateTo));
                         RecoveryReceiptResponse response = recoveryService.recoverReceiptKOByCI(ci, dateFrom, dateTo);

@@ -76,3 +76,3 @@
                         log.debug("Invoking API operationrecoverReceiptKOForCreditorInstitution - args: {} {} {} {}",
-                                sanitizeInput(ci), sanitizeInput(iuv), dateFrom, dateTo);
+                                sanitizeInput(ci), sanitizeInput(iuv), sanitizeInput(dateFrom), dateTo);

@@ -92,4 +92,5 @@
                     // build the service bus message
-                    ServiceBusMessage serviceBusMessage = new ServiceBusMessage(message.toString());
-                    log.debug("Sending scheduled message {} to the queue: {}", message.toString(), queueName);
+                    String sanitizedMessage = sanitizeInput(message.toString());
+                    ServiceBusMessage serviceBusMessage = new ServiceBusMessage(sanitizedMessage);
+                    log.debug("Sending scheduled message {} to the queue: {}", sanitizedMessage, queueName);
@@ -100,3 +101,3 @@
                     // log event
-                    log.debug("Sent scheduled message_base64 {} to the queue: {}", LogUtils.encodeToBase64(message.toString()), queueName);
+                    log.debug("Sent scheduled message_base64 {} to the queue: {}", LogUtils.encodeToBase64(sanitizedMessage), queueName);
                     generateRE(InternalStepStatus.RPT_TIMER_CREATED, "Scheduled RPTTimerService: [" + message + "]");

@@ -27,2 +27,4 @@
             import static it.gov.pagopa.wispconverter.util.CommonUtility.sanitizeInput;
+            import java.util.regex.Pattern;
+            import java.util.regex.Matcher;
             import static it.gov.pagopa.wispconverter.util.MDCUtil.setRPTTimerInfoInMDC;
@@ -100,3 +102,4 @@
                     // log event
-                    log.debug("Sent scheduled message_base64 {} to the queue: {}", LogUtils.encodeToBase64(message.toString()), queueName);
+                    String sanitizedMessage = sanitizeInput(message.toString());
+                    log.debug("Sent scheduled message_base64 {} to the queue: {}", LogUtils.encodeToBase64(sanitizedMessage), queueName);
                     generateRE(InternalStepStatus.RPT_TIMER_CREATED, "Scheduled RPTTimerService: [" + message + "]");
@@ -107,2 +110,11 @@
+                private String sanitizeInput(String input) {
+                    if (input == null) {
+                        return null;
+                    }
+                    // Remove any control characters and newlines
+                    Pattern pattern = Pattern.compile("[\\p{Cntrl}\\n\\r]");
+                    Matcher matcher = pattern.matcher(input);
+                    return matcher.replaceAll("");
+                }

@@ -53,3 +53,3 @@
                     try {
-                        log.debug("Invoking API operationcreateRPTTimer - args: {}", request.toString());
+                        log.debug("Invoking API operationcreateRPTTimer - args: {}", sanitizeInput(request.toString()));
                         rptTimerService.sendMessage(request);

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

chore: editing logs configuration [PAGOPA-2221] #130

chore: editing logs configuration [PAGOPA-2221] #130

FedericoRuzzier commented Oct 2, 2024

github-actions bot commented Oct 2, 2024

github-actions bot commented Oct 2, 2024

Provide additional feedback

Please help us improve GitHub Copilot by sharing more details about this comment.

Provide additional feedback

Please help us improve GitHub Copilot by sharing more details about this comment.

Provide additional feedback

Please help us improve GitHub Copilot by sharing more details about this comment.

Provide additional feedback

Please help us improve GitHub Copilot by sharing more details about this comment.

github-actions bot commented Oct 17, 2024

sonarcloud bot commented Oct 17, 2024

Provide additional feedback

Please help us improve GitHub Copilot by sharing more details about this comment.

Provide additional feedback

Please help us improve GitHub Copilot by sharing more details about this comment.

Provide additional feedback

Please help us improve GitHub Copilot by sharing more details about this comment.

Provide additional feedback

Please help us improve GitHub Copilot by sharing more details about this comment.

Provide additional feedback

Please help us improve GitHub Copilot by sharing more details about this comment.

Provide additional feedback

Please help us improve GitHub Copilot by sharing more details about this comment.

Provide additional feedback

Please help us improve GitHub Copilot by sharing more details about this comment.

		log.debug("Invoking API operation recoverReceiptKOForCreditorInstitution - args: {} {} {}",
		sanitizeInput(ci), dateFrom, dateTo);

		log.debug("Invoking API operation recoverReceiptKOForCreditorInstitution - args: {} {} {} {}",
		sanitizeInput(ci), sanitizeInput(iuv), dateFrom, dateTo);

@@ -108,3 +108,4 @@
                     try {
-                        log.debug("Invoking API operationreceiptKo - args: {}", request);
+                        String sanitizedRequest = sanitizeInput(request);
+                        log.debug("Invoking API operationreceiptKo - args: {}", sanitizedRequest);
                         receiptService.sendKoPaaInviaRtToCreditorInstitution(List.of(mapper.readValue(request, ReceiptDto.class)).toString());

@@ -56,3 +56,3 @@
                     try {
-                        log.debug("Invoking API operationcreateTimer - args: {}", request.toString());
+                        log.debug("Invoking API operation createTimer - args: {}", sanitizeRequest(request));
                         receiptTimerService.sendMessage(request);
@@ -95,2 +95,10 @@
                 }
+                private String sanitizeRequest(ReceiptTimerRequest request) {
+                    return String.format("ReceiptTimerRequest(sessionId=%s, fiscalCode=%s, noticeNumber=%s, expirationTime=%d)",
+                            request.getSessionId(),
+                            request.getFiscalCode(),
+                            request.getNoticeNumber(),
+                            request.getExpirationTime());
+                }
             }

@@ -81,3 +81,3 @@
                     try {
-                        log.debug("Invoking API operationdeleteTimer - args: {}", sanitizeInput(paymentTokens));
+                        log.debug("Invoking API operation deleteTimer");
                         List<String> tokens = Arrays.asList(paymentTokens.split(","));

@@ -28,2 +28,9 @@
+            private String sanitizeInput(String input) {
+                if (input == null) {
+                    return null;
+                }
+                return input.replaceAll("[\\r\\n]", "_");
+            }
             @RestController
@@ -104,3 +111,3 @@
                     try {
-                        log.debug("Invoking API operationrecoverReceiptToBeReSent - args: {}", request.toString());
+                        log.debug("Invoking API operationrecoverReceiptToBeReSent - args: {}", sanitizeInput(request.toString()));
                         return ResponseEntity.ok(recoveryService.recoverReceiptToBeReSent(request));
@@ -123,3 +130,3 @@
                     try {
-                        log.debug("Invoking API operationrecoverReceiptToBeReSentByPartition - args: {}", request.toString());
+                        log.debug("Invoking API operationrecoverReceiptToBeReSentByPartition - args: {}", sanitizeInput(request.toString()));
                         return ResponseEntity.ok(recoveryService.recoverReceiptToBeReSentByPartition(request));

chore: editing logs configuration [PAGOPA-2221] #130

Are you sure you want to change the base?

chore: editing logs configuration [PAGOPA-2221] #130

Conversation

FedericoRuzzier commented Oct 2, 2024

List of Changes

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):

Types of changes

Checklist:

github-actions bot commented Oct 2, 2024

github-actions bot commented Oct 2, 2024

github-actions bot commented Oct 17, 2024

sonarcloud bot commented Oct 17, 2024

Quality Gate failed